Add export project interaction details page #7438
Draft
+7,762
−5,434
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
instead of calling getMessages() arbitrarily
paulgain
changed the base branch from
main
to
feature/export-project-interactions
paulgain
changed the title
data-hub-frontend
|
||||||||||||||||||||||||||||
| Project |
data-hub-frontend
|
| Branch Review |
feature/export-interaction-details
|
| Run status |
|
| Run duration | 01m 40s |
| Commit |
|
| Committer | Edmond De Los Reyes |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
Failures
|
0
|
Flaky
|
0
|
Pending
|
0
|
Skipped
|
0
|
Passing
|
15
|
| View all changes introduced in this branch ↗︎ | |
paulgain
force-pushed
the
feature/export-interaction-details
branch
from
adfc563 to
81129a5
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## feature/export-project-interactions #7438 +/- ##
=======================================================================
+ Coverage 88.55% 88.58% +0.02%
=======================================================================
Files 1161 1146 -15
Lines 18041 17806 -235
Branches 5129 5098 -31
=======================================================================
- Hits 15977 15773 -204
+ Misses 2064 2033 -31 ☔ View full report in Codecov by Sentry. |
…r be an empty string
Replace `format` with `formatDate` and clean up unused code
Update test date format to match date format used in component
Bumps [@chromatic-com/storybook](https://github.com/chromaui/addon-visual-tests) from 3.2.2 to 3.2.3. - [Release notes](https://github.com/chromaui/addon-visual-tests/releases) - [Changelog](https://github.com/chromaui/addon-visual-tests/blob/main/CHANGELOG.md) - [Commits](chromaui/addon-visual-tests@v3.2.2...v3.2.3) --- updated-dependencies: - dependency-name: "@chromatic-com/storybook" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [react-to-print](https://github.com/MatthewHerbst/react-to-print) from 3.0.2 to 3.0.4. - [Release notes](https://github.com/MatthewHerbst/react-to-print/releases) - [Changelog](https://github.com/MatthewHerbst/react-to-print/blob/master/CHANGELOG.md) - [Commits](MatthewHerbst/react-to-print@v3.0.2...v3.0.4) --- updated-dependencies: - dependency-name: react-to-print dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [react-router-dom](https://github.com/remix-run/react-router/tree/HEAD/packages/react-router-dom) from 7.1.0 to 7.1.1. - [Release notes](https://github.com/remix-run/react-router/releases) - [Changelog](https://github.com/remix-run/react-router/blob/main/packages/react-router-dom/CHANGELOG.md) - [Commits](https://github.com/remix-run/react-router/commits/[email protected]/packages/react-router-dom) --- updated-dependencies: - dependency-name: react-router-dom dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [elastic-apm-node](https://github.com/elastic/apm-agent-nodejs) from 4.9.0 to 4.10.0. - [Release notes](https://github.com/elastic/apm-agent-nodejs/releases) - [Changelog](https://github.com/elastic/apm-agent-nodejs/blob/main/CHANGELOG.asciidoc) - [Commits](elastic/apm-agent-nodejs@v4.9.0...v4.10.0) --- updated-dependencies: - dependency-name: elastic-apm-node dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Chore/dependencies 2025 01 02
* WIP: Add overseas region metadata and filter Co-authored-by: Samuele Mattiuzzo <[email protected]>
…vestments on company activity page (#7415) * Display EYB activity card and EYB label on linked investments on company activity page --------- Co-authored-by: Marijn Kampf <[email protected]>
The error occurs because we're invoking history.pushState too frequently, which exceeds the browser's limit of 100 calls per 10 seconds. The onPageClick is being called repeatedly in a short time frame due to a re-render loop which can trigger this error.
…push-state Fix excessive calls to `window.history.pushState`
Sort a default company export interaction list
Is a best practice to declare constant functions in the beginning of the codebase before calling, in order the javascript interpreter can locate.
dredmonds
force-pushed
the
feature/export-interaction-details
branch
from
7c75407 to
d2487e8
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of change
This PR introduces an export project interaction details page. Currently, there is no backend endpoint available for this page, which will need to be implemented separately. In the interim, this PR uses a locally imported JSON file that mirrors the expected structure of the future endpoint's response.
Once the endpoint is implemented, follow the straightforward instructions provided in
src/client/modules/ExportPipeline/ExportInteractionDetails/index.jsxto replace the JSON file with the API response.Component tests will need to be written after the endpoint is implemented.
Test instructions
Go to:
/export/<export-uuid>/interactions/detailsScreenshots
Checklist